Skip to content

Conversation

luigidellaquila
Copy link
Contributor

Fixing non-snapshot telemetry, after we promoted FORK for release, see #129606

Fixes: #129888

@luigidellaquila luigidellaquila requested a review from astefan June 25, 2025 07:56
@luigidellaquila luigidellaquila added >test Issues or PRs that are addressing/adding tests test-release Trigger CI checks against release build :Analytics/ES|QL AKA ESQL labels Jun 25, 2025
@elasticsearchmachine elasticsearchmachine added Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) v9.1.0 labels Jun 25, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-analytical-engine (Team:Analytics)

Copy link
Contributor

@astefan astefan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@luigidellaquila luigidellaquila enabled auto-merge (squash) June 25, 2025 08:33
@luigidellaquila
Copy link
Contributor Author

Also this #129839 didn't account for release tests

@luigidellaquila luigidellaquila disabled auto-merge June 27, 2025 13:48
@luigidellaquila luigidellaquila merged commit 0299f4e into elastic:main Jun 27, 2025
30 of 35 checks passed
mridula-s109 pushed a commit to mridula-s109/elasticsearch that referenced this pull request Jul 3, 2025
ioanatia pushed a commit to ioanatia/elasticsearch that referenced this pull request Jul 7, 2025
ioanatia added a commit that referenced this pull request Jul 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Analytics/ES|QL AKA ESQL Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >test Issues or PRs that are addressing/adding tests test-release Trigger CI checks against release build v9.2.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CI] XPackRestIT test {p0=esql/60_usage/Basic ESQL usage output (telemetry) non-snapshot version} failing

3 participants